[BA-2211] Update the SDK constructor to accept the new originVerification property#35
Conversation
- domain verification key generation script for well known file - yarn generate-key-script - readme doc on how to use it
packages/client/src/MWPClient.ts
Outdated
There was a problem hiding this comment.
hmm we probably don't need this func if we have MWPClient.request({ method: 'wallet_getNonce' })?
packages/client/src/MWPClient.ts
Outdated
There was a problem hiding this comment.
let's use the name domainVerification instead. It's more consistent with how we're talking about this feature in other places
packages/client/src/MWPClient.ts
Outdated
There was a problem hiding this comment.
I don't think we actually need to define it here. A developer won't need to wallet_getNonce directly. Rather our SDK will call our own nonce endpoint and pass it into the generateDomainSignature function that the developer supplies
There was a problem hiding this comment.
i think it makes sense to have these log in the script tho, maybe we can disable this rule for /scripts/ folder?
There was a problem hiding this comment.
I had to remove them because they kept failing the CI
3 participants
Summary
https://linear.app/coinbase/issue/BA-2211/wsdk-update-the-sdk-constructor-to-accept-the-new-originverification
Added the optional originVerification property to the MWPClient sdk, as well as the EIP1193Provider, and the wagmi constructor. Made a dummy getNonce function (which we will replace to call the actual API) that returns a uuid. Wrote some testing code for it in the MWPClient.test.ts file.
How did you test your changes?
Added testing code for it in the MWPClient.test.ts and ran yarn test to ensure everything passed.